Update influencer fields of Security ML integrations - #20335
Conversation
Elastic Docs Style Checker (Vale)Summary: 1 suggestion found 💡 Suggestions (1): Optional style improvements. Apply when helpful.
The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
|
Pinging @elastic/sec-applied-ml (Team:Security-Applied ML) |
| field: user.id | ||
| script: | ||
| source: "return doc.containsKey('user.name') && !doc['user.name'].isEmpty() ? doc['user.name'].value : ''" | ||
| lang: painless |
There was a problem hiding this comment.
Can we use missing_bucket: true for these instead?
There was a problem hiding this comment.
Added this option instead of the script
| field: network.transport | ||
| 'process.executable': | ||
| terms: | ||
| field: process.executable |
There was a problem hiding this comment.
One other thing I picked up, if you are using Network Packet Capture as the data source, do you need to enable the monitor_processes setting in the integration policy?
https://github.com/elastic/integrations/blob/63384ae307ea445e86c064c37c84eebf24815e97/packages/network_traffic/_dev/build/docs/README.md#monitor_processes
There was a problem hiding this comment.
Good find, added this under a note in README
|
Tick the box to add this pull request to the merge queue (same as
|
|
✅ All changelog entries have the correct PR link. |
💚 Build Succeeded
History
|
|
Package ded - 3.4.0 containing this change is available at https://epr.elastic.co/package/ded/3.4.0/ |
|
Package dga - 3.2.0 containing this change is available at https://epr.elastic.co/package/dga/3.2.0/ |
|
Package lmd - 3.4.0 containing this change is available at https://epr.elastic.co/package/lmd/3.4.0/ |
|
Package pad - 2.4.0 containing this change is available at https://epr.elastic.co/package/pad/2.4.0/ |
|
Package problemchild - 3.2.0 containing this change is available at https://epr.elastic.co/package/problemchild/3.2.0/ |
Proposed commit message
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Using ITP
Related issues
Screenshots